Skip to content

Add Denmark country data and public holidays#169

Open
hjertis wants to merge 12 commits into
openpotato:developfrom
hjertis:develop
Open

Add Denmark country data and public holidays#169
hjertis wants to merge 12 commits into
openpotato:developfrom
hjertis:develop

Conversation

@hjertis

@hjertis hjertis commented Jul 6, 2026

Copy link
Copy Markdown

Introduce Denmark (DK) into the dataset by adding country and language registry entries, a new DK subdivision file with the five regions, and national public holiday records for 2025-2035 with Danish, English and German names.

Introduce Denmark (DK) into the dataset by adding country and language registry entries, a new DK subdivision file with the five regions, and national public holiday records for 2025-2035 with Danish and English names.
Copilot AI review requested due to automatic review settings July 6, 2026 07:01

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request introduces Denmark (DK) to the dataset by registering Denmark in the country/language registries, adding a DK subdivision listing, and adding national public holidays for 2025–2035 with Danish/English names.

Changes:

  • Added a Denmark entry to src/countries.csv.
  • Added a Danish language entry to src/languages.csv.
  • Added DK subdivisions data and DK public holiday records (2025–2035).

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
src/languages.csv Adds a new language registry entry intended for Danish.
src/countries.csv Adds Denmark to the country registry and declares its official language(s).
src/dk/subdivision.csv Adds DK subdivision data (currently not aligned with existing subdivision file conventions).
src/dk/holidays/holidays.public.csv Adds Denmark national public holidays for 2025–2035 with DA/EN names.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/languages.csv
Comment thread src/countries.csv Outdated
Comment thread src/dk/subdivisions.csv Outdated
hjertis and others added 3 commits July 6, 2026 10:15
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Updates `countries.csv` to use `DA` instead of `DK` for Denmark, and renames `src/dk/subdivision.csv` to `src/dk/subdivisions.csv` to match the expected plural filename convention.

@hjertis hjertis left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be fixed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

Comment thread src/countries.csv Outdated
Comment thread src/dk/subdivisions.csv Outdated
Comment thread src/dk/holidays/holidays.public.csv Outdated
Comment thread src/countries.csv Outdated
hjertis and others added 3 commits July 9, 2026 09:47
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Adds Denmark to the README country coverage list and updates DK public holiday records to include German (DE) translations alongside Danish and English names for 2025–2035. It also migrates `src/dk/subdivisions.csv` to the standardized subdivision schema (country/ISO codes, multilingual category/name fields, official language, and parent columns) for consistency with the repository format.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 13 comments.

Comment thread README.md Outdated
Comment thread src/dk/subdivisions.csv Outdated
Comment thread src/dk/holidays/holidays.public.csv Outdated
Comment thread src/dk/holidays/holidays.public.csv Outdated
Comment thread src/dk/holidays/holidays.public.csv Outdated
Comment thread src/dk/holidays/holidays.public.csv Outdated
Comment thread src/dk/holidays/holidays.public.csv Outdated
Comment thread src/dk/holidays/holidays.public.csv Outdated
Comment thread src/dk/holidays/holidays.public.csv Outdated
Comment thread src/dk/holidays/holidays.public.csv Outdated
hjertis and others added 3 commits July 10, 2026 06:48
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
- Fix German spelling: 'Gruendonnerstag' → 'Gründonnerstag' in public holidays
- Add German translations for all 5 Danish regions in subdivisions
- Update README to reflect Denmark has only public holidays (not school holidays)

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Comment thread src/dk/subdivisions.csv Outdated
Comment thread src/dk/holidays/holidays.public.csv
Comment thread src/dk/holidays/holidays.public.csv
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Comment thread src/dk/subdivisions.csv Outdated
Reorder the Denmark subdivision records in `src/dk/subdivisions.csv` so the region codes are listed in ascending order.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants